Conversation
Remove commissionable node browsing from the device controller. It is currently not used and causes additional mDNS traffic.
marcelveldt
left a comment
There was a problem hiding this comment.
We have no idea if this is not in use with any of the lib consumers. There are other projects nowadays depending on our server. Is it really needed to remove this ? I can't imagine this causes that much traffic.
If it is, we need to deprecate it first by logging a warning for a couple of months and then remove it.
|
AND/OR you introduce a feature toggle with variant
Having that toggle in place might also facilitate measuring the actual mdns traffic delta in an otherwise identical setup |
Yeah I was considering something like that, e.g. we could only start active browsing when However, it seems that browsing is actually switching to a passive mode after 4 tries. Having a continuous browse doesn't seem as bad as I initially thought. Given that, and the fact that we might have users using this, let's close this. |
Remove commissionable node browsing from the device controller. It is currently not used and causes additional mDNS traffic.